label: Don't set out parameters twice

We set them later again when we calculate the vertical size, so just
move the general assignment to -1 into the horizontal branch.

widget: Remove useless NULL check

We are checking for allocation != NULL at the beginning of the function
already. This also means that the allocation parameter is not nullable.

roundedbox: Initialize the hashtable key before using it

In mem_hash, we are iterating over the individual bytes of the key, so
make sure all of it is initialized.

cssimagebuiltin: Remove icons that don't draw anything anymore

cssimagebuiltin: Remove unused variables

widget: Remove useless comment

wayland: fix up/down mix up in discrete events

The wayland specification for discrete step information for scroll and
other axes reads:

| The discrete value carries the directional information. e.g. a
| value of -2 is two steps towards the negative direction of this axis.

mutter sets a value of 1 for SCROLL_DOWN events and -1 for SCROLL_UP
events.

gdkdevice Wayland backend does the opposite, it translates a positive
discrete value as SCROLL_UP and a negative value as SCROLL_DOWN, which
ends up inverting the scrolling direction.

Fix the logic in gdkdevice Wayland to use a positive value as
SCROLL_DOWN and a negative value as SCROLL_UP so that it matches mutter
and weston logic.

https://bugzilla.gnome.org/show_bug.cgi?id=765907

css: Stop drawing double borders for builtins

The fallback code for rendering builtin checks, radios
and expanders was using border parameters. With the generic
gadget borders using the same parameters, this was giving
double borders.

shadowsvalue: Don't draw invisible box shadows

defaultvalue test: Skip GdkMonitor::geometry

We never return NULL for ::geometry and ::workarea. Just skip
these properties.

GdkMonitor: Fix refresh-rate property

We were using g_value_set_boolean for an integer property.

Use default VFS, not a local one (it has no network)

Fix gtk_paned_set_position to set position_set consistently

As pointed out in https://bugzilla.gnome.org/show_bug.cgi?id=765676,
we were failing to set position_set if the position did not change.

testnotebookdnd: add one GtkListBox as tab content

To make sure we test regressions on widgets that bubble up motion
handling and can trigger the motion handling code path on GtkNotebook.

https://bugzilla.gnome.org/show_bug.cgi?id=764395

gtknotebook: avoid tab dnd from content

Before commit 6c1bee2377eba8 we were setting an attribute of GtkNotebook
to track the pressed button if the pressed button happened on the
tab itself.
Later in the motion handling code we were checking whether the private
pressed button attribute was set or not in order to handle a tab dnd
or not.

In commit 6c1bee2377eba8 the code changed and set the pressed button
variable unconditionally, which means, a motion event from within the
tab content triggered a tab reordering.

This happened only if the children hierarchy have a widget that bubbles
up both button press event, which sets the private pressed button
attribute; and motion events, which started the tab dnd checking the
private pressed button attribute.
A widget that experienced the regression was GtkListBox.

In order to fix it, set the button pressed variable only when it press
the tab itself, not the content.

https://bugzilla.gnome.org/show_bug.cgi?id=764395

GtkPlacesView: check for network:// URI support before using it

Specifically, this URI is not supported on Windows, but GFile will
do its "best" and turn it into GLocalFile("$pwd/network"), with
spectacularly bad results.

https://bugzilla.gnome.org/show_bug.cgi?id=765858

gdkevents: Free motion events discarded by motion compression

Those should be freed together with their list link.

dnd: Make sure to free a stashed event

I am not sure if this happens in practice, but better to be safe.

gesture: Don't leak stashed events

These events were never freed.

Avoid copying an event

Not a big deal, but we can be a bit more efficient when generating
grab-broken events.

Small docs improvement

Cross reference from gdk_event_get_keycode to
gdk_event_get_scancode.

Drop unused macros

These have been sitting here forever, unused. Time to drop them

x11: drop a useless list

We were keeping all the event sources in a list, only to remove
them at the end of their life. Not useful.

wayland: drop a useless list

We were keeping all the event sources in a list, only to remove
them at the end of their life. Not useful.

Fix the build

I forgot to add an include here.

wayland: Tag the event source with the acutal display name

wayland: Make gdk_display_get_name work

It should just return the same as gdk_screen_make_display_name.

css: Use setters for font size and family

This avoid creating pango font descriptions all the time, when
the gtk-font-name setting hasn't actually changed.

settings: Add setters for font family and size

treeview: Use getter for gtk-enable-animations

switch: Use getter for gtk-enable-animations

stack: Use getter for gtk-enable-animations

scrolled window: Use getter for gtk-enable-animations

Use getter for gtk-enable-animations

popover: Use getter for gtk-enable-animations

Use a variable consistently

Use a better hash for arcs

css: Use a getter for the enable-animations setting

dnd: Use a getter for the drag threshold

Add a few settings getters

Trivial whitespace fix

Use the new getter

This avoids the use of generic api in the per-event code path.

gdk: Add a private getter for GdkDevice::input-mode

Add a forgotten deprecation

gdk_display_get_screen is marked as deprecated in the docs.

W32: support cairo library in non-standard locations

When checking for cairo_win32_surface_create_with_format in -lcairo,
temporarily put CAIRO_LIBS into LIBS so that AC_CHECK_LIB() can
find it in weird places like /usr/local/lib, where gcc would not look
on its own.

https://bugzilla.gnome.org/show_bug.cgi?id=765793

Updated Scottish Gaelic translation

W32: reword the error about required cairo version

1.14.x series does not contain the functionality needed by GTK+.
Advise to use 1.15.x (1.15.2 is available) or 1.16.x (yet to be released).

https://bugzilla.gnome.org/show_bug.cgi?id=765790

settings: Cache xsettings

Instead of calling out to gdk every time an XSetting is requested,
cache the value (we already have the property_values array
anyway).

https://bugzilla.gnome.org/show_bug.cgi?id=765579

box gadget: Redo expand flag handling

We only keep one align flag per child, so it seems odd to
keep separate h/v expand flags. Just keep one expand flag
and interpret it according to orientation. Allow setting
the expand flag for child widgets too, though, so we can
make widget expand without interfering with the recursive
widget expand flag.

Update all callers.

Use the new possibility of expanding child widgets to make
the label of check and radio buttons expand. This fixes
unexpected behavior of these widgets in RTL in some places.

https://bugzilla.gnome.org/show_bug.cgi?id=765742

expander: Reverse alignments in RTL

Flip alignments of the title gadget children.

check button: Reverse alignments in RTL

Flip alignments for the box gadget children.

box gadget: allow reversing alignments

In RTL, we want to interpret GTK_ALIGN_START and _END
in the opposite way. Since we don't give gadgets a text
direction, just allow setting an align_reverse flag
to the box gadget.

Adwaita: refactor levelbar styling.

broadway: Fix include rules in gdkbroadwaymonitor.h

Fixes 044109c8c73f5c1d9eb470c857112c08a005b0ae

Updated Spanish translation

window: Stop using gdk_screen_get_n_monitors

settings: Stop comparing screen numbers

There is only one screen.

window: Stop using screen width/height

treeview: Stop using screen width/height

Popups should always be placed relative to a monitor.

label: Stop using screen width/height

Popups should always be placed relative to a monitor.

combobox: Stop using screen width/height

Popups should always be placed relative to a monitor.

gtk: Port to new monitor api

Use the GdkDisplay monitor api instead of the GdkScreen one.

menu: Add new monitor api

This will work better with the new monitor api in gdk.

inspector: Port to new monitor api

Adapt to new monitor apis, and show more information.

win32: Port to new monitor api

x11: Bring back fullscreen-on-all-monitors support

It is unfortunate the EWMH requires Xinerama here - lets do this
without the caching that was previously there, I don't think it is
needed.

x11: Port to new monitor api

x11: Drop Xinerama

It may come back after the monitor conversion.

broadway: Port to new monitor api

wayland: Port to new monitor api

integrate docs

screen: Implement old monitor apis generically

Implement all the monitor-related GdkScreen apis based on GdkMonitor.

Add a fallback for unconverted backends

If the monitor vfuncs are not implemented in a display class,
fall back to providing a single monitor object representing
the entire screen. This is not meant to be 'good enough', it
is just to provide some implementation until all backends
implement the monitor vfuncs. When that is the case, the
fallback should be removed.

display: Add new monitor apis

This follows our general direction of moving functionality
from GdkScreen to GdkDisplay.

Add GdkMonitor

Add a simple object representing a monitor.

Revert "iconhelper: check if surface was invalidated"

This reverts commit 88d02583974659d1d37d825df78c79bee4c95d0e.

GtkTreeView: change child repositioning to preserve locality

If there was a piece of text in the cell, then when the edit
entry is shown for that cell, it should have a piece of text
in it roughly at the same location.

Therefore, when child widget is enlarged (child preferred
size exceeds cell size), extra width should be added by
extending either left or right edge depending on text direction.

If after that the child sticks outside of the treeivew visible region,
try to push it back inside (breaking its alignment with the
cell), again, giving preference (i.e. adjusting it last)
to either left or right edge depending on text direction.

https://bugzilla.gnome.org/show_bug.cgi?id=765471

iconhelper: check if surface was invalidated

Fetching the style may cause the surface to be invalidated.

Updated Hungarian translation

Updated Spanish translation

Adwaita: add a background to cellrendertext

3.21.1

Updates

wayland: Ignore NoSymbol keys

NoSymbol is not a valid GDK symbol (it only has the concept of
VoidSymbol, for some reason, which is neither the same thing nor
produced by any sane keymap). Passing NoSymbol events through to GTK+
apps is unlikely to produce anything useful.

In particular, this meant VTE would scroll to the end of the buffer when
pressing Fn (required for Page Up/Down on Macs), as it was receiving a
keypress that wasn't a modifeir. This does not happen on X11, as the
KEY_FN keycode is above 255, so does not get sent to clients.

https://bugzilla.gnome.org/show_bug.cgi?id=764825

placesview: update a leftover function name

This should've been part of the previous commit.

placesview: follow up fixes

This patch unmarks the addresses' prefixes as translatable,
and updates the callback name to reflect what it really does
now.

This is a follow up fix from commit 7449add82.

https://bugzilla.gnome.org/show_bug.cgi?id=756570

wayland: Perform seat grab focus checks on native windows

We don't care about the specific (possibly client-side) window that
requested the focus here, only the toplevel. Fixes mistakenly sent
focus events when the grab happens inside the current focus window.

https://bugzilla.gnome.org/show_bug.cgi?id=762756

pixelcache: reuse existing timeout source when possible

This avoids the g_source_remove(), g_source_destroy(),
g_timer_source_new(), and g_source_set_name_by_id() in the common case.

Instead, we reuse our previous source and update the ready time to our
new deadline. We lose the coalescing with g_timeout_add_seconds(), but
that is not going to help in the common case anyway (unless you have
three hands and can scroll multiple pixelcached backed widgets at once).

https://bugzilla.gnome.org/show_bug.cgi?id=765640

Updated Spanish translation

dnd: Fix some issues with drag icons

In non-composited environments, we were ending up with all-black
drag icons, because nothing was drawing the background of our new
toplevel. Fix this by drawing background when we are not composited.
We don't do this when composited, since we want to allow transparent
icons.

Fix indentation

dnd: Set attributes from all widgets for text drag icons

There is nothing textview-specific here.

placesview: provide guidance on server addresses

GtkPlacesView currently provides no example of
server addresses, which may confuse users.

To fix that, add a helper popover with some
guidance on server addresses.

https://bugzilla.gnome.org/show_bug.cgi?id=756570

progressbar: Enforce invariants

When measuring gadgets, minimum must come out <= natural.
Make sure we don't fail this when measuring progressbar text.

https://bugzilla.gnome.org/show_bug.cgi?id=765644

dnd: Fix lifecycle issues with widgets as drag icons

The documentation clearly says that the widget is not destroyed,
but we were in fact failing to keep it alive, since it was still
a child or the icon_window when we destroy that. Fix this by
reparenting the icon_widget out before. Also, deal with the
possibility that the application might destroy the widget
halfway through, for whatever reason.

testdnd2: Actually test gtk_drag_set_icon_widget

I added a new test function, but didn't actually use it.
No wonder I couldn't reproduce the lifecycle issues with
drag widgets that firefox is experiencing.

docs: typo in GTK_STYLE_CLASS_LIST_ROW docs

Fix typo in documentation of GTK_STYLE_CLASS_LIST_ROW: s/rowss/rows/g

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=765632

gdk/dnd: Don't use default display when getting cursor

Always associate a drag context with a GdkDisplay and use that when
getting a cursor for a given action.

If we don't do this, dragging on a window that doesn't use the default
display will make us use cursors from the wrong display.

https://bugzilla.gnome.org/show_bug.cgi?id=765565

frametimings: reuse previous frame timing in common case

Typically, there won't be any references on old frame timings except for
the most recent timing. So instead of discarding these and re-entering
gslice twice, just steal the old frame timing and reuse it.

https://bugzilla.gnome.org/show_bug.cgi?id=765592

gdk/dnd: Add missing signal parameter documentation

The cancel reason parameter was missing.

https://bugzilla.gnome.org/show_bug.cgi?id=765577